Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] - 6차 데모데이 1차 QA(시모) #578

Merged
merged 4 commits into from
Oct 24, 2024
Merged

Conversation

simorimi
Copy link

@simorimi simorimi commented Oct 23, 2024

✅ 작업 내용

  • 체크박스 아이콘 보이지 않는 문제 해결
  • 여행기 보러 가기 클릭시 루트 페이지로 이동하는 문제 해결
  • 태그 에니메이션 자동 드레그 문제 해결
  • 태그 에니메이션 최초의 위에 있다가 내려오는 문제 해결

📸 스크린샷

🙈 참고 사항

  • 태그 에니메이션 최초의 위에 있다가 내려오는 문제 해결
    해당 문제의 경우 chip 내부의 animation때문에 발생하는 것으로 알게 되었습니다. 다만, 당최 이유는 모르겠네요. 애초에 에니메이션의 경우 아래에서 원래 위치로 등장하게 했으나 왜 검색 페이지로 갔다가 홈으로 돌아오면 원래 위치보다 위에 있다가 아래로 떨어지는지 잘 모르겠습니다.
    우선, 해당 부분 해결은 에니메이션을 반대로 적용했습니다. 위에서 원래위치로 등장하게 에니메이션을 적용하였고 이로인하여 해결할 수 있었습니다. 우선에 이렇게 동작하도록 하고 추후 해당 부분 개선을 고민해봐야겠습니다.

@simorimi simorimi added this to the sprint 6 milestone Oct 23, 2024
@simorimi simorimi self-assigned this Oct 23, 2024
@simorimi simorimi changed the title [Fix] 6차 데모데이 1차 QA(시모) [Fix] - 6차 데모데이 1차 QA(시모) Oct 23, 2024
Copy link

github-actions bot commented Oct 23, 2024

Test Results

11 tests  ±0   11 ✅ ±0   14s ⏱️ ±0s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit b460e61. ± Comparison against base commit 8017c22.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@jinyoung234 jinyoung234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

작업 고생 많으셨습니다!

Copy link

@0jenn0 0jenn0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다 시모

@jinyoung234 jinyoung234 merged commit c8102c1 into develop/fe Oct 24, 2024
3 checks passed
@simorimi simorimi deleted the fix/fe/#577 branch November 21, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants